Skip to content

Find StyleSheet declarations that are not variables #223

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

draperunner
Copy link
Contributor

Fixes #214

Instead of only looking for StyleSheet declarations in VariableDeclarator nodes, we check if CallExpressions are StyleSheet.create({...}). This handles the export default StyleSheet.create({...}) case and other cases where the style sheet is not assigned to a variable.

@coveralls
Copy link

coveralls commented Mar 14, 2019

Coverage Status

Coverage decreased (-0.2%) to 77.336% when pulling 8921b20 on draperunner:sort-styles-default-export into 2a280bb on Intellicode:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 74.903% when pulling a81ab29 on draperunner:sort-styles-default-export into 89c3a11 on Intellicode:master.

@Intellicode
Copy link
Owner

Thanks for your PR! If you resolve the conflicts I think we can merge

@draperunner draperunner force-pushed the sort-styles-default-export branch from d6cc5d1 to 875c030 Compare March 28, 2019 09:44
@draperunner draperunner force-pushed the sort-styles-default-export branch from 875c030 to 8921b20 Compare March 28, 2019 09:57
@draperunner
Copy link
Contributor Author

@Intellicode I have fixed the merge conflicts.

@Intellicode Intellicode merged commit 54bae56 into Intellicode:master Apr 3, 2019
@Intellicode
Copy link
Owner

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants